Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace dependency management plugin with Gradle platform dependencies #298

Merged
merged 1 commit into from
Jun 25, 2020

Conversation

royclarkson
Copy link
Member

No description provided.

@royclarkson royclarkson requested a review from spikymonkey June 19, 2020 17:18
@codecov
Copy link

codecov bot commented Jun 19, 2020

Codecov Report

Merging #298 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #298   +/-   ##
=========================================
  Coverage     91.83%   91.83%           
  Complexity     1202     1202           
=========================================
  Files           146      146           
  Lines          2904     2904           
  Branches        294      294           
=========================================
  Hits           2667     2667           
  Misses          176      176           
  Partials         61       61           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64171cf...6126273. Read the comment docs.

@royclarkson royclarkson force-pushed the gradle-platform-dependencies branch from 979fe76 to 99b4141 Compare June 23, 2020 13:28
@royclarkson royclarkson force-pushed the gradle-platform-dependencies branch from 99b4141 to 6126273 Compare June 24, 2020 21:35
Copy link

@dyroberts dyroberts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Long overdue proper bom support!
Glad to see gradle finally making this a reality.
To me it seems like a cleaner implementation, as well.
Maybe we should get this in to all our projects sooner than later?

@@ -18,12 +18,14 @@ import org.springframework.cloud.contract.verifier.config.TestFramework

buildscript {
dependencies {
classpath("io.spring.gradle:dependency-management-plugin:1.0.9.RELEASE")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious, why use the dm plugin here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spring Cloud Contract seems to have some dependency on the DM plugin. When I tried to use the platform dependency, the submodule would not build. Considering this is only for tests and we aren't publishing this module, I decided to leave it alone.

@royclarkson royclarkson merged commit aaf5bda into master Jun 25, 2020
@royclarkson royclarkson deleted the gradle-platform-dependencies branch June 25, 2020 17:20
@royclarkson royclarkson added this to the 3.2.0-M2 milestone Aug 4, 2020
@royclarkson royclarkson modified the milestones: 3.2.0-M2, 3.2.0-RC1 Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants